-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add OVN IC CI #1338
Add OVN IC CI #1338
Conversation
🤖 Created branch: z_pr1338/dfarrell07/ovn_ic_ci_pr |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Worked fine, using=ovn-ic , Thanks
@mkolesnik can you please review? |
685ec28
to
42ce623
Compare
Fixes: submariner-io/enhancements#209 Co-authored-by: Mike Kolesnik <[email protected]> Signed-off-by: Daniel Farrell <[email protected]>
42ce623
to
c4fbe0d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but based on #1297 (comment) perhaps you want to always rebuild OVN images, at least until they fix it..
It doesn't have to be part of this patch but since you're already changing this area might as well..
If not, then it should probably be done either way after this gets merged.
Let's do it after we rebase the kind 0.20 bump on to this and test both the rebuilt image and the old image. |
🤖 Closed branches: [z_pr1338/dfarrell07/ovn_ic_ci_pr] |
Fixes: submariner-io/enhancements#209